All four phase routes now render ContinuousWorkspaceView, which shows
all realized phase sections stacked in one scroll surface. Unrealized
future phases are skipped. Only the current reachable phase section
has an actionable bottom artifact. Graph view is unchanged.

InterviewView is preserved as-is for its existing test coverage.

Co-authored-by: Amp <amp@ampcode.com>

One controller owns the single chat session, lifecycle hooks, auto-phase-intent,
and per-phase section projection. ContinuousWorkspaceView no longer double-reads
from useSpecificationBundleData + useInterviewController — it uses only the new
controller. useInterviewController is preserved unchanged for InterviewView tests.

Co-authored-by: Amp <amp@ampcode.com>

Added WorkspaceFocusContext to bridge scroll position from the continuous
workspace view to the phase navigation sidebar. IntersectionObserver on
section divs updates focusedPhase; the sidebar uses it for is-active
highlighting instead of route-active matching when available. Route-based
highlighting is preserved as fallback for graph view and export routes.

Co-authored-by: Amp <amp@ampcode.com>

Co-authored-by: Amp <amp@ampcode.com>

… core

Move 9 pure helper functions (live activity, tool items, reasoning text,
turn reference comparison) and the bottom-artifact enrichment chain from
both useInterviewController and useContinuousWorkspaceController into
-interview-controller-core.ts. Both controllers now import from core
instead of carrying character-identical copies.

Also moves InterviewControllerBottomArtifactState type to core (re-exported
from -interview-controller.ts for downstream consumers) and fixes
ContinuousWorkspaceController to use InterviewDurableSpecificationState
directly instead of ReturnType<typeof ...>.

Net reduction: ~185 lines. No behavior change.

Co-authored-by: Amp <amp@ampcode.com>

Remove dead InterviewView and useInterviewController mocks from
router.test.tsx (no production route renders them anymore). Rename
interviewView* counters to workspaceView* to match the actual component
under test (ContinuousWorkspaceView). Remove provenance comment from
continuous controller.

Completes Step 4 of the CONTINUOUS_WORKSPACE_HYBRID.md migration plan.
The route-first assumptions were already structurally retired by Steps
1-2 (continuous controller derives activePhase from workflow state, not
route params). This commit cleans up the residual test artifacts.

Co-authored-by: Amp <amp@ampcode.com>

Clear stale workspace focus on route changes and stabilize continuous workspace section inputs so scroll-spy observers are not rebuilt on unrelated renders.

Co-authored-by: Cursor <cursoragent@cursor.com>

Update mutation callback refs synchronously so stable action wrappers always call the latest mutation implementation.

Co-authored-by: Cursor <cursoragent@cursor.com>

Avoid rebuilding the continuous workspace observer for artifact-only section changes by keying the effect to the rendered phase list instead of stabilizing controller artifacts.

Co-authored-by: Cursor <cursoragent@cursor.com>

Keep submitted-phase live tool indicators running so the refactored controllers preserve the previous pre-stream UI behavior.

Co-authored-by: Cursor <cursoragent@cursor.com>

Re-applies FE-705's a4827a2 'Fix completed tool activity rendering'
to the extracted -interview-controller-core helpers, so the live
tool-running indicator goes false the moment a tool returns
output-available instead of persisting until the chat stream ends.

InterviewView fixture for the 'pre-stream generating window' test
now seeds the tool part with state: 'input-available' (the actual
pre-stream state), which is what hasRunningLiveTool requires.

HANDOFF.md is volatile session-transfer state — its own contract said
"Delete this file after PR is submitted." ln-sync and ln-build skills
both instruct agents to remove stale HANDOFF.md files rather than
preserve them as archive breadcrumbs.